Skip to content

feat: Reintroduce crowdin synchronization. #4122

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 0 commits into from
Closed

Conversation

IMB11
Copy link
Member

@IMB11 IMB11 commented Aug 5, 2025

This is a continuation of @brawaru's previous pull request before the monorepo (modrinth/knossos#1640) with the main change being it only runs on the main branch, rather than using Crowdin's branching functionality to have multiple branches of sources, which we wouldn't use anymore.

Thank you @brawaru!


Prerequisites

  • Delete all sources from the Crowdin project (whilst keeping the translation memory!)
  • Make sure there are no github-related integrations in the integrations tab.
  • Create a fine-grained access token for the "Modrinth Bot" github account with the following permissions, placing it in the CROWDIN_GH_TOKEN repository environment secret;
    • contents: write
    • pull-requests: write
  • Copy the https://crowdin.com/project/modrinth project ID into the CROWDIN_PROJECT_ID repository variable.
  • Create an access token on the "Modrinth" crowdin account with all scopes, and place the token into CROWDIN_PERSONAL_TOKEN repo secret.
  • Modify the pull request description template in .github as needed (probably wont need any changes honestly)
  • Merge the PR.
  • The push workflow should run and push the new sources to Crowdin.
  • (optional) Pre-translate using translation memory.

The first pull request made by the CI will be large as it has to populate the initial locale files with {}, after that it should be pretty small and review-able.

@IMB11 IMB11 added frontend DevEx Improvements to developer experience labels Aug 5, 2025
@IMB11
Copy link
Member Author

IMB11 commented Aug 5, 2025

After #4025 is merged, we can add it to the crowdin.yml for the moderation package. Can be done in a different PR though.

@brawaru

This comment was marked as resolved.

@IMB11 IMB11 requested review from Prospector and triphora August 5, 2025 16:29
@IMB11 IMB11 enabled auto-merge August 11, 2025 16:03
@brawaru

This comment was marked as resolved.

@IMB11

This comment was marked as resolved.

@brawaru

This comment was marked as resolved.

@IMB11
Copy link
Member Author

IMB11 commented Aug 14, 2025

We've reverted the deletion, will rename the files.

@IMB11 IMB11 closed this Aug 15, 2025
auto-merge was automatically disabled August 15, 2025 12:21

Pull request was closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DevEx Improvements to developer experience frontend
Development

Successfully merging this pull request may close these issues.

2 participants